Skip to content

Conversation

georgewallace
Copy link
Contributor

No description provided.

@georgewallace
Copy link
Contributor Author

@shainaraskas I think this is good for a v1. let me know what you think.

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

95% there. just some questions re: applies_to, some big images, and some questions re: things that might not apply to serverless. generally, I see the following things as possible serverless red flags:

  • settings written.like.this and passed in config files/panels
  • 3p integrations like terraform
  • APIs

Comment on lines 33 to 35
## Connector networking configuration [_connector_networking_configuration]

Use the [action configuration settings](https://www.elastic.co/guide/en/kibana/current/alert-action-settings-kb.html#action-settings) to customize connector networking configurations, such as proxies, certificates, or TLS settings. You can set configurations that apply to all your connectors or use `xpack.actions.customHostSettings` to set per-host configurations.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't apply to serverless - what do serverless users do?

Comment on lines 66 to 69
:::{image} ../images/kibana-connector-select-type.png
:alt: Connector select type
:class: screenshot
:::
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this image is vv large - can we use the width attribute to resize?

image


For out-of-the-box and standardized connectors, refer to [preconfigured connectors](https://www.elastic.co/guide/en/kibana/current/pre-configured-connectors.html).

::::{tip}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this feels like it isn't applicable to serverless either

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are correct. I took a stab at calling out both preconfigured connectors and the Terraform provider in a singular note. Not sure if this is ok with our current guidelines.

Comment on lines 83 to 86
:::{image} ../images/kibana-connectors-import-banner.png
:alt: Connectors import banner
:class: screenshot
:::
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this image is even more large

consider the value of the images: do we need them?

## Monitoring connectors [monitoring-connectors]

**This page is a work in progress.** The documentation team is working to combine content pulled from the following pages:
The [Task Manager health API](../deploy-manage/monitor/kibana-task-manager-health-monitoring.md) helps you understand the performance of all tasks in your environment. However, if connectors fail to run, they will report as successful to Task Manager. The failure stats will not accurately depict the performance of connectors.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this API isn't available in serverless.


* [/raw-migrated-files/kibana/kibana/action-types.md](/raw-migrated-files/kibana/kibana/action-types.md)
* [/raw-migrated-files/docs-content/serverless/action-connectors.md](/raw-migrated-files/docs-content/serverless/action-connectors.md)
For more information on connector successes and failures, refer to the [Event log index](../explore-analyze/alerts-cases/alerts/event-log-index.md).
Copy link
Collaborator

@shainaraskas shainaraskas Feb 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if this applies (or not sure how to access without the API)

:class: screenshot
:::

## Monitoring connectors [monitoring-connectors]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shainaraskas I took a stab at going into a bit more details here to talk about monitor in both cases with the Task manager health API and with the event log. Since you can use the event log in both

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some final polish stuff re: the presentation of context-sensitive info

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: !

@georgewallace georgewallace merged commit 31304e9 into elastic:main Feb 20, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants